-
-
Notifications
You must be signed in to change notification settings - Fork 7.6k
test: add CTest to cmake to enable testing #2753
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
This pull request has been automatically marked as abandoned because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Helps with #2753 Co-authored-by: realstealthninja <[email protected]>
@vil02 Do you think we should wait until all test cases pass, i.e. fix all the files? |
In short: yes. But the optimistic thing is that maybe one can work directory by directory, i.e. select one, let's say Doing all at once is way too much. |
Helps with TheAlgorithms#2753 Co-authored-by: realstealthninja <[email protected]>
Description of Change
Testing
This pr enables testing of the algorithms in the repository
before we get to the stage of running tests on a CI. We will need to remove all
other code from every main function so that the only remaining code is a call to the test function.
for now this draft pull request shows how easy it is to enable testing.
adding it to the awesome CI will be as easy as adding
make test to a test step.
Checklist
Notes: